-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Guest OS rules #10098
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Guest OS rules #10098
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #10098 +/- ##
============================================
+ Coverage 17.66% 17.70% +0.03%
- Complexity 14063 14109 +46
============================================
Files 5307 5309 +2
Lines 473814 473854 +40
Branches 55637 55602 -35
============================================
+ Hits 83715 83901 +186
+ Misses 380774 380636 -138
+ Partials 9325 9317 -8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
4835d26 to
796ecb1
Compare
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
|
@blueorangutan package |
|
@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
# Conflicts: # api/src/main/java/com/cloud/host/Host.java # ui/src/views/infra/HostUpdate.vue
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
Co-authored-by: Fabricio Duarte <[email protected]>
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
…method # Conflicts: # server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
Description
Currently, ACS provides a feature that enables a guest OS preference for a specific host, prioritizing VMs based on its OS. However, this functionality allows VMs with different OSes to be deployed in a host with a Guest OS defined, as expected as it is only a preference configuration; this is not always desired. For example, keeping proprietary OS in the same host could reduce licensing costs.
This PR tackles this scenario, allowing JavaScript rules, using the JS interpreter introduced in #5909, to direct VMs to specific hosts based on their OS. Different from the guest OS preference, these guest OS rules will be strict. This new feature added a new field,
Guest OS as JS rulein the host edit view to allow users the creation of these rules.This PR depends on #9074, for this reason it is on draft until #9074 is merged.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Screenshots (if appropriate):
How Has This Been Tested?
In a local lab with two hosts (
host-1andhost-2) I created the guest OS rulevmGuestOs.toLowerCase().indexOf("debian") != -1forhost-2, and deployed multiple VMs using templates and ISOs.host-2was filtered during the deployment.host-2and verified that it was not possible.host-1tohost-2and vice versa.How did you try to break this feature and the system with this change?
I tried to deploy VMs in the same lab without the guest OS rules defined and validated that the deployment of VMs were being done as expected.